Skip to content

765 added subject headings search link#914

Open
kamerynB wants to merge 1 commit intorelease-v1.68from
765-add-subject-headings-search-link
Open

765 added subject headings search link#914
kamerynB wants to merge 1 commit intorelease-v1.68from
765-add-subject-headings-search-link

Conversation

@kamerynB
Copy link
Copy Markdown
Contributor

No description provided.

@kamerynB kamerynB requested a review from clarkepeterf April 13, 2026 20:44
@clarkepeterf
Copy link
Copy Markdown
Contributor

There is an error running this locally because some subject headings don't have IDs. The frontend should handle undefined / null IDs.

http://localhost:3000/view/text/4d37d47b-2e2a-4769-b82e-c662e9be5060

@kamerynB kamerynB force-pushed the 765-add-subject-headings-search-link branch from 9dec7f1 to 3e571d4 Compare April 21, 2026 15:21
@kamerynB kamerynB force-pushed the 765-add-subject-headings-search-link branch from 3e571d4 to af22e2c Compare April 21, 2026 15:22
@kamerynB
Copy link
Copy Markdown
Contributor Author

There is an error running this locally because some subject headings don't have IDs. The frontend should handle undefined / null IDs.

http://localhost:3000/view/text/4d37d47b-2e2a-4769-b82e-c662e9be5060

@clarkepeterf I have fixed this issue. I also added one more feature where the Show all X results links change their text when in mobile. They become Show all X work results. The reason why is that the headers for the columns are removed in mobile and this will provide more context to what the user will be searching.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants